-
Notifications
You must be signed in to change notification settings - Fork 219
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(fix) O3-4004: Fix Patient action menu button workspace #1336
Conversation
…ns (#1294) This PR refactors the registration form's cancel modal to match new modal naming and registration conventions. Modals are now registered in the routes registry file under the `modals`. The naming convention has also changed - modals now use the `*.modal.tsx` suffix. I've also amended the modal to use Carbon's ModalBody, ModalHeader, and ModalFooter components instead of using divs with custom classes. Finally, I've amended the modal title and content to align with other confirmation modals in O3.
import { launchWorkspace } from '@openmrs/esm-framework'; | ||
import { type WardPatient, type WardPatientWorkspaceProps } from '../types'; | ||
|
||
let wardPatient: WardPatient = null; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a comment on why we need this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added the comment.Thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Bhargav. While I don't love this fix (as the underlying issue is still there), it does unblock us. Approved with minor comment.
We should also keep the ticket open, but we can de-prioritize it and move it out of the v1.0 epic (probably into v1.5).
Requirements
Summary
The Patient action menu button does not work when other action menus are clicked and again when we click back the patient action button. Hopefully this pr fixed it.
Screenshots
OpenMRS.and.4.more.pages.-.Personal.-.Microsoft_.Edge.2024-10-06.19-51-33.webm
Related Issue
https://openmrs.atlassian.net/browse/O3-4004